Skip to content

testing the exe of main.py#176

Closed
DagaBhai wants to merge 7 commits into
pyfenn:mainfrom
DagaBhai:temptesting
Closed

testing the exe of main.py#176
DagaBhai wants to merge 7 commits into
pyfenn:mainfrom
DagaBhai:temptesting

Conversation

@DagaBhai

@DagaBhai DagaBhai commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

for #136
test_check_all_templates_dir
Verifies that all expected template directories are present in the templates folder and that no unexpected template directories exist
test_check_for_main
Ensures every template contains a main.py file, which serves as the template's entry point
test_check_for_requirement
Ensures every template contains a requirements.txt file listing its Python dependencies
test_main
Validates template executability by installing the template's dependencies, running its main.py script, and then uninstalling the dependencies before proceeding to the next template

Refactor template directory path handling in tests.
@DagaBhai
DagaBhai marked this pull request as ready for review July 4, 2026 05:25
@DagaBhai

DagaBhai commented Jul 4, 2026

Copy link
Copy Markdown
Contributor Author

@blkdmr ig its working, the job is failing because multiple templates are missing requirements.txt files or their main.py
also the fenn repository (the parent repository) is currently pointing to an older commit of the templates submodule.

@blkdmr

blkdmr commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Hi @DagaBhai, looks like you are still working on this. Can i merge this PR? If your tests have found some bugs or inconsistencies, can you open some dedicated issues?

@DagaBhai

DagaBhai commented Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

Hi @DagaBhai, looks like you are still working on this. Can i merge this PR? If your tests have found some bugs or inconsistencies, can you open some dedicated issues?

pr is done for template testing and i have added it in the workflow also but pls see this make sure the all test pass can you pls fix these as idk how to do these

the job is failing because multiple templates are missing requirements.txt files or their main.py also the fenn repository (the parent repository) is currently pointing to an older commit of the templates submodule.

@blkdmr

blkdmr commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

@ApusBerliozi please look at this when you have time. If is okay for you, we can merge this pr

@ApusBerliozi

Copy link
Copy Markdown
Contributor

Hey @DagaBhai !

Could you please check, why tests are failing?

@ApusBerliozi

Copy link
Copy Markdown
Contributor

Hey @DagaBhai! Kindly resolve conflicts

@DagaBhai DagaBhai closed this Jul 15, 2026
@DagaBhai
DagaBhai deleted the temptesting branch July 15, 2026 09:55
@DagaBhai

Copy link
Copy Markdown
Contributor Author

ill raise new pr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants